-
Notifications
You must be signed in to change notification settings - Fork 274
feat: *ByRole a11y value option #1210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov ReportBase: 94.67% // Head: 94.82% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1210 +/- ##
==========================================
+ Coverage 94.67% 94.82% +0.15%
==========================================
Files 42 43 +1
Lines 2946 3036 +90
Branches 440 454 +14
==========================================
+ Hits 2789 2879 +90
Misses 157 157
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
52007ea
to
a7e8ff6
Compare
373d4be
to
dd27483
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very good ! I just added a comment on some naming that I find a bit confusing and a smaill typo in docs
afe8402
to
dafbcb2
Compare
Summary
value
option to*ByRole
query*ByA11yValue
query by allowingtext
entry to be string or regexTest plan
Added relevant tests for
*ByRole
query and*ByA11yValue
.